Skip to content

fix: official letter openings not masked as PIB (v2.6.9)#42

Merged
click0 merged 1 commit into
mainfrom
claude/refactor-data-masking-lIcWN
Jul 7, 2026
Merged

fix: official letter openings not masked as PIB (v2.6.9)#42
click0 merged 1 commit into
mainfrom
claude/refactor-data-masking-lIcWN

Conversation

@click0

@click0 click0 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Повідомляємо Вам was masked as a PIB

In Повідомляємо Вам, що відповідно до пункту 15 розділу XII Інструкції … the parser saw two Title-Case words in a row and masked them as "Surname Name" → Повритулляємо Володимир, що ….

Fix

  • looks_like_name rejects words with 1st/2nd-person-plural verb endings (-ємо, -имо, -емо, -єте, -ите, -ете) — no Ukrainian name/surname ends this way.
  • Pronouns Вам/Вами/Ваш(-а/-е) and common official verbs (Повідомляємо, Просимо, Направляємо, Надаємо) added to EXCLUDE_WORDS (only Вас was there before).
  • Real PIB after such phrases is still masked: Повідомляємо, що сержант Коваленко Марія Іванівна прибула → phrase kept, rank+PIB masked.

Test plan

  • 461 tests pass (+3: reported sentence untouched, more official phrases, PIB-after-phrase still masked)

https://claude.ai/code/session_01XT6iUWaQgahXDB9TWX9Bq7


Generated by Claude Code

'Повідомляємо Вам, що ...' was treated as 'Surname Name'. Reject
1st/2nd-person-plural verb endings (-ємо/-имо/-емо/-єте/-ите/-ете) in
looks_like_name; add pronouns Вам/Вами/Ваш and common official verbs to
EXCLUDE_WORDS. Real PIB after such phrases still masked. +3 tests.

https://claude.ai/code/session_01XT6iUWaQgahXDB9TWX9Bq7
@click0 click0 merged commit cc36bd1 into main Jul 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants